Skip to content

fix: collect PyPI metrics weekly alongside GitHub metrics#73

Merged
amrit110 merged 2 commits into
mainfrom
fix/collect-pypi-metrics-weekly
Mar 15, 2026
Merged

fix: collect PyPI metrics weekly alongside GitHub metrics#73
amrit110 merged 2 commits into
mainfrom
fix/collect-pypi-metrics-weekly

Conversation

@amrit110
Copy link
Copy Markdown
Member

Summary

  • requests was missing from pyproject.toml, causing collect_pypi_metrics.py to silently fail (it has continue-on-error: true) — PyPI data has been stale since 2025-12-03
  • collect-metrics.yml was using uv pip install --system pyyaml instead of uv sync, so the lock file wasn't being used
  • PyPI metrics were only collected as a side effect of deploy-catalog-gcp.yml, not on a dedicated weekly schedule

Changes

  • Add requests>=2.32.0 to pyproject.toml core dependencies and update uv.lock
  • Switch collect-metrics.yml to use uv sync for dependency installation
  • Use uv run python consistently for all script invocations in the workflow
  • Add a "Collect PyPI metrics" step to the weekly Monday collect-metrics.yml workflow, staging both GitHub and PyPI JSON files in the same commit

Test plan

  • PR checks pass
  • After merge, verify collect-metrics.yml workflow run succeeds and commits updated pypi_metrics.json with a fresh timestamp

amrit110 and others added 2 commits March 15, 2026 19:29
- Add `requests` to pyproject.toml dependencies (was missing, causing
  silent failures in collect_pypi_metrics.py)
- Switch `collect-metrics.yml` from `uv pip install --system pyyaml`
  to `uv sync` to use the lock file properly
- Use `uv run python` consistently for all script invocations
- Add PyPI metrics collection step to the weekly Monday schedule,
  committing both github and pypi metric files together

PyPI data was last updated 2025-12-03 due to `requests` not being
installed during the collect-metrics workflow run.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@amrit110 amrit110 merged commit 72bac25 into main Mar 15, 2026
5 checks passed
@amrit110 amrit110 deleted the fix/collect-pypi-metrics-weekly branch March 15, 2026 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant